Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add async api #461

Merged
merged 3 commits into from
Dec 17, 2024
Merged

feat: add async api #461

merged 3 commits into from
Dec 17, 2024

Conversation

th4s
Copy link
Contributor

@th4s th4s commented Sep 6, 2024

This PR adds an async api:

  • implement Sink and Stream for WebRtcChannel
  • add type RawPeerChannel with AsyncRead and AsyncWrite support
  • add functions take_raw and take_raw_by_id on WebRtcSocket

Closes: #453

@th4s th4s changed the title Add async api feat: add async api Sep 6, 2024
Copy link
Owner

@johanhelsing johanhelsing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really useful. And like you say, an easy win.

Sorry for the slow response. We are most likely removing the typestate pattern from the matchbox socket so might need an update if #468 gets in first.

@johanhelsing
Copy link
Owner

#468 is in. If you fix the conflicts, we'll get this in in time for the 0.11 release

@th4s
Copy link
Contributor Author

th4s commented Dec 16, 2024

wait a second. I think I introduced a bug...

@th4s
Copy link
Contributor Author

th4s commented Dec 16, 2024

should be good now 👍

@johanhelsing
Copy link
Owner

Thanks, sorry if it sounded like I was demanding something. I really appreciate your help and totally respect people being busy or having lower capacity at times (otherwise I'd be a total hypocrite :))

@johanhelsing johanhelsing merged commit 750c9d9 into johanhelsing:main Dec 17, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement AsyncWrite and AsyncRead for WebRtcChannel
2 participants